Skip to content

feat(agent-manager): show reviewer avatars in PR sidebar - #13952

Merged
marius-kilocode merged 5 commits into
mainfrom
add-reviewer-avatar-to-pr-sidebar
Sep 9, 2026
Merged

marius-kilocode merged 5 commits into
mainfrom
add-reviewer-avatar-to-pr-sidebar

Conversation

@marius-kilocode

@marius-kilocode marius-kilocode commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

What Problem This Solves

The Agent Manager PR sidebar showed reviewer names with generic state icons and a separate Review: Approved overview row. It did not show reviewer profile pictures or present each review state in the GitHub-style reviewer list.

Why This Change Was Made

  • Show each reviewer's GitHub avatar beside their login.
  • Move the review state to a right-aligned status icon with an explanatory tooltip.
  • Use GitHub-like icons and colors for approved, changes requested, commented, and pending states.
  • Remove the redundant Review: Approved overview row while retaining the aggregate decision in PR Summary.
  • Enrich reviewer data with avatar URLs from the GraphQL response and cache successful lookups by login, including resolved no-avatar results.
  • Include avatar URLs in the status signature so recovered images update the webview.

User Impact

The PR sidebar now makes the reviewer identity and review state visible at a glance. Approved reviewers show a profile picture and green check, changes requested reviewers show a red x, commented reviewers show a comment icon, and pending reviewers show a neutral pending icon.

Evidence

Before

Reviewer sidebar before the update

After

Reviewer sidebar after the update

The after screenshot is the generated Agent Manager visual-regression capture. The before image is a reference reconstruction of the supplied original sidebar screenshot.

Validation

  • bun run compile in packages/kilo-vscode/
  • bun run typecheck
  • bun run lint
  • bun run test:unit, 5460 passed, 2 skipped
  • Focused poller and signature tests, 185 passed
  • Isolated VS Code self-test verified the rendered PR sidebar

Comment thread packages/kilo-vscode/src/agent-manager/PRStatusPoller.ts Outdated
@kilo-code-bot

kilo-code-bot Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (12 files)
  • .changeset/pr-reviewer-avatars.md
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/agentmanager/pr-panel-reviewers-chromium-linux.png
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/agentmanager/pr-panel-summary-chromium-linux.png
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/labs-tool-call-lab/search-previews-chromium-linux.png
  • packages/kilo-vscode/src/agent-manager/PRStatusPoller.ts
  • packages/kilo-vscode/src/agent-manager/pr/am-pr-utils.ts
  • packages/kilo-vscode/tests/unit/am-pr-status-bridge.test.ts
  • packages/kilo-vscode/tests/unit/am-pr-utils.test.ts
  • packages/kilo-vscode/webview-ui/agent-manager/pr/PROverview.tsx
  • packages/kilo-vscode/webview-ui/agent-manager/pr/PRReviewers.tsx
  • packages/kilo-vscode/webview-ui/agent-manager/pr/pr-panel.css
  • packages/kilo-vscode/webview-ui/src/stories/agent-manager.stories.tsx
Previous Review Summaries (3 snapshots, latest commit fb17960)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit fb17960)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (11 files)
  • .changeset/pr-reviewer-avatars.md
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/agentmanager/pr-panel-reviewers-chromium-linux.png
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/agentmanager/pr-panel-summary-chromium-linux.png
  • packages/kilo-vscode/src/agent-manager/PRStatusPoller.ts
  • packages/kilo-vscode/src/agent-manager/pr/am-pr-utils.ts
  • packages/kilo-vscode/tests/unit/am-pr-status-bridge.test.ts
  • packages/kilo-vscode/tests/unit/am-pr-utils.test.ts
  • packages/kilo-vscode/webview-ui/agent-manager/pr/PROverview.tsx
  • packages/kilo-vscode/webview-ui/agent-manager/pr/PRReviewers.tsx
  • packages/kilo-vscode/webview-ui/agent-manager/pr/pr-panel.css
  • packages/kilo-vscode/webview-ui/src/stories/agent-manager.stories.tsx

Previous review (commit dec40ce)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (10 files)
  • .changeset/pr-reviewer-avatars.md
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/agentmanager/pr-panel-reviewers-chromium-linux.png
  • packages/kilo-vscode/src/agent-manager/PRStatusPoller.ts
  • packages/kilo-vscode/src/agent-manager/pr/am-pr-utils.ts
  • packages/kilo-vscode/tests/unit/am-pr-status-bridge.test.ts
  • packages/kilo-vscode/tests/unit/am-pr-utils.test.ts
  • packages/kilo-vscode/webview-ui/agent-manager/pr/PROverview.tsx
  • packages/kilo-vscode/webview-ui/agent-manager/pr/PRReviewers.tsx
  • packages/kilo-vscode/webview-ui/agent-manager/pr/pr-panel.css
  • packages/kilo-vscode/webview-ui/src/stories/agent-manager.stories.tsx

Previous review (commit c7ac518)

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-vscode/src/agent-manager/PRStatusPoller.ts 339 Incomplete avatar cache can refetch every poll, and recovered avatars may never reach the UI
Files Reviewed (8 files)
  • .changeset/pr-reviewer-avatars.md
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/agentmanager/pr-panel-reviewers-chromium-linux.png
  • packages/kilo-vscode/src/agent-manager/PRStatusPoller.ts - 1 issue
  • packages/kilo-vscode/tests/unit/am-pr-status-bridge.test.ts
  • packages/kilo-vscode/webview-ui/agent-manager/pr/PROverview.tsx
  • packages/kilo-vscode/webview-ui/agent-manager/pr/PRReviewers.tsx
  • packages/kilo-vscode/webview-ui/agent-manager/pr/pr-panel.css
  • packages/kilo-vscode/webview-ui/src/stories/agent-manager.stories.tsx

Fix these issues in Kilo Cloud


Reviewed by grok-4.6 · Input: 111.1K · Output: 15K · Cached: 1M

Review guidance: REVIEW.md from base branch main

@marius-kilocode
marius-kilocode force-pushed the add-reviewer-avatar-to-pr-sidebar branch from c7ac518 to faace61 Compare September 9, 2026 09:39
@marius-kilocode
marius-kilocode force-pushed the add-reviewer-avatar-to-pr-sidebar branch from dec40ce to 200dd86 Compare September 9, 2026 11:14
@marius-kilocode
marius-kilocode force-pushed the add-reviewer-avatar-to-pr-sidebar branch from fb17960 to cbb0d53 Compare September 9, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants